Usr/sogh/mcpconfigentities.v3 #2976
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… Usr/sogh/dyn-tool-reg
… Usr/sogh/dyn-tool-reg
… Usr/sogh/dyn-tool-reg
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 6 pipeline(s). |
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 6 pipeline(s). |
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 6 pipeline(s). |
- Added IgnoreMember for RuntimeOptions.Mcp in ModuleInitializer - Regenerated all 4 ConfigurationTests verified snapshots without Mcp section - All ConfigurationTests now pass locally (4/4) - Mcp configuration properly excluded from snapshot testing
…mparison issues - Removed the 'mcp' section from dab-config.MsSql.json test configuration file - This prevents the Mcp configuration from being deserialized and appearing in test snapshots - The IgnoreMember setting in ModuleInitializer.cs was not fully preventing Mcp property serialization - Removing Mcp from source config file ensures clean snapshot comparisons - All 4 ConfigurationTests (MsSql, MySql, PostgreSql, Cosmos) now passing (0 failed, 4 passed) Resolves MS SQL pipeline test failure in TestReadingRuntimeConfigForMsSql where: - UserProvidedAllTools was showing as 'true' in received vs 'false' in verified - Stored procedure GraphQL Operation was showing as 'Mutation' vs expected 'Query'
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 6 pipeline(s). |
…ization - Reordered Entity.cs properties from (GraphQL, Rest, Mcp, Permissions) to (Rest, GraphQL, Mcp, Permissions) - Updated constructor parameters to match property order - Fixed test files using old constructor parameter order - Accepted snapshots with corrected property ordering This ensures System.Text.Json serialization produces consistent output, preventing snapshot test failures in pipelines where deserialization-reserialize cycles would produce different property ordering.
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 6 pipeline(s). |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why make this change?
#syntax. e.g. Closes #XXWhat is this change?
How was this tested?
Sample Request(s)